-
Notifications
You must be signed in to change notification settings - Fork 111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[745] send moderate vacancy #768
[745] send moderate vacancy #768
Conversation
@fey есть несколько вопросов по жизненному циклу вакансии 1 может ли пользователь сам отправлять вакансию в архив |
@usernaimandrey давай сейчас придерживаться текущего поведения, т.е. 1-2 - нет. 3 - сейчас да. |
@fey тогда можно поревьюить |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
В целом всё хорошо. Заапрувлю.
Единственно пожалуй чего не хватает это прямого теста для действия которое добавляется ПР-ом - собственно отправка вакансии на модерацию. Докиньте, пожалуйста.
f48285e
to
071a042
Compare
@corsicanec82 can you review? |
Сразу хочу сказать, что зависимость dev + минорное обновление не должно ломать обратную совместимость, но тест из за нее на пайпе падал |
lgtm |
No description provided.